Copy the phpmd and phplint configs into the project directory #195

becw · 2022-10-24T15:04:01Z

Copy the phpmd and phplint configs into the project directory, the same way as the phpcs config.

The goal here is to make it more obvious that phpmd and phplint are part of the code-review process of any particular project, and make it easier for project teams to customize those checks.

Todo

Should each of the checks be broken out into its own phing target? (which would also be run by the code-review target?) Unlike phpcs, phpmd and phplint can't get all of their parameters from their config files.

# Picks up on phpcs.xml:
$> phpcs
$> phpcbf

# Phing provides <phplint> task that accepts a <fileset> argument, which makes it much
# easier to run php linting on a large set of files. On the command line, you can use
# PHP's built in linter against one file at a time:
# @see https://www.phing.info/guide/hlhtml/#PhpLintTask
$> php -l FILENAME

# PHPmd requires the directories, output format, ruleset file, and file suffixes be passed
# as arguments:
$> phpmd docroot/modules/custom,docroot/themes/custom text phpmd.xml --suffixes=php,inc,module,theme,profile,install,test

…me way as the phpcs config.

Copy the phpmd and phplint configs into the project directory, the sa…

4d9d742

…me way as the phpcs config.

becw added the WIP - do not merge label Oct 24, 2022

fix phplint.txt destination filename during install

7d4cb29

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Copy the phpmd and phplint configs into the project directory #195

Copy the phpmd and phplint configs into the project directory #195

becw commented Oct 24, 2022 •

edited

Loading

Copy the phpmd and phplint configs into the project directory #195

Are you sure you want to change the base?

Copy the phpmd and phplint configs into the project directory #195

Conversation

becw commented Oct 24, 2022 • edited Loading

Todo

becw commented Oct 24, 2022 •

edited

Loading